Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BC Wallet deep link display #421

Merged
merged 7 commits into from
Mar 4, 2024
Merged

BC Wallet deep link display #421

merged 7 commits into from
Mar 4, 2024

Conversation

loneil
Copy link
Contributor

@loneil loneil commented Feb 16, 2024

Add capability of displaying a deep link specific to the BC Wallet app. On devices that can have the app (iOS, Android) show the Deep link as the main option, with an ability to get a QR code as well.
See Kim's mockups here for UI/UX https://xd.adobe.com/view/b40869d2-5fff-40ff-855c-ace0d1da0c30-e15b/

Additionally, change the flow to generate the presentation request message contents when the controller loads (and creates the QR code and deep link), rather than waiting until the short QR link is called back to.
This message content is stored with the auth_session document in the mongo collection, so that it is fetched instead of generated on that QR callback.

@loneil loneil force-pushed the feature/bcWalletDeepLink branch from bd87625 to 61a80b1 Compare February 23, 2024 23:44
@loneil loneil marked this pull request as ready for review February 27, 2024 21:23
@loneil loneil changed the title Draft: BC Wallet deep link display BC Wallet deep link display Feb 27, 2024
esune
esune previously approved these changes Feb 27, 2024
Copy link
Member

@esune esune left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good based on what was reviewed with @loneil .
Once merged we can create a pre-release for dev and test in a live environment.

@esune esune merged commit db6eaff into main Mar 4, 2024
2 checks passed
@esune esune deleted the feature/bcWalletDeepLink branch March 4, 2024 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants